Skip to content

Conversation

@arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Nov 7, 2024

PR-Codex overview

This PR focuses on improving the handling of the nonce property in the transaction object to ensure it is only assigned when it is explicitly defined.

Detailed summary

  • Changed the assignment of nonce to check if "nonce" exists in transaction and if it is not undefined.
  • If both conditions are met, nonce is assigned from transaction.nonce; otherwise, it defaults to null.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@arcoraven arcoraven force-pushed the ph/handleUndefinedNonce branch from f418baa to f420ede Compare November 7, 2024 23:35
@arcoraven arcoraven merged commit 1ba738a into main Nov 7, 2024
3 checks passed
@arcoraven arcoraven deleted the ph/handleUndefinedNonce branch November 7, 2024 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants